Update installation scripts and modernize package management#15
Merged
Conversation
- chrome: Replace deprecated apt-key with modern GPG keyring approach (/etc/apt/keyrings/) for Debian compatibility - codex: Auto-install Node.js 22.x when not available instead of failing - gemini-cli: Auto-install Node.js 22.x when not available instead of failing - yek: Copy binary to /usr/local/bin when installer places it in ~/.local/bin (not in PATH) - mcp-language-server: Update test scenario Go image from 1.21 to 1.24 (mcp-language-server v0.1.1 requires Go >= 1.24.0) https://claude.ai/code/session_014AiNJx7n8PhgNNwqhAUTzR
- Add test/codex/test.sh and test/gemini-cli/test.sh for autogenerated tests (devcontainer features test requires these files) - Fix npm permission error: always install as root since npm's global directory (/usr/lib/node_modules/) requires root permissions. The previous su to _REMOTE_USER caused EACCES errors on non-root images. https://claude.ai/code/session_014AiNJx7n8PhgNNwqhAUTzR
- chrome: 1.0.1 -> 1.0.2 (fix deprecated apt-key) - codex: 1.0.0 -> 1.0.1 (auto-install Node.js, fix npm permissions) - gemini-cli: 1.0.0 -> 1.0.1 (auto-install Node.js, fix npm permissions) - yek: 1.0.2 -> 1.0.3 (fix PATH issue for installed binary) https://claude.ai/code/session_014AiNJx7n8PhgNNwqhAUTzR
Replace deprecated Debian bullseye-based images with bookworm in codex and gemini-cli test scenarios. https://claude.ai/code/session_014AiNJx7n8PhgNNwqhAUTzR
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR modernizes several installation scripts to improve reliability and security, while also updating test infrastructure to use current base images.
Key Changes
Chrome installation: Replaced deprecated
apt-keyapproach with modern GPG keyring management in/etc/apt/keyrings/, following current Debian/Ubuntu best practicesNode.js dependency handling: Updated Codex CLI and Gemini CLI installers to automatically install Node.js 22.x if not present, rather than failing with an error. This improves user experience by removing the need for manual prerequisite installation
Yek binary placement: Added logic to copy the yek binary from
~/.local/binto/usr/local/binto ensure it's globally available in PATH after installationTest infrastructure: Updated Go devcontainer image from
1.21-bullseyeto1.24-bookwormto use a current Go version and modern base OSImplementation Details
signed-byparameter to specify the GPG key location, which is the recommended approach for secure package managementhttps://claude.ai/code/session_014AiNJx7n8PhgNNwqhAUTzR